[WPB-27169] Do not count apps and collaborators as members in get-team-size. by fisx · Pull Request #5445 · wireapp/wire-server

fisx · 2026-08-13T13:43:36Z

This is a follow-up to #5173, #5213, #5452, where we added data about number of apps and collaborators to the get-team-size response.

Changes:

The teamSize field works again as it did in the olden days, only counting team members, which are regular users by definition.
There are two more fields apps, collaborators, which contain the resp. counts.
Now all 3 counts add up to the legalhold limit, which is enforced correctly again.
The protobuf is also adjusted. I went for backwards-incompatible, because the change I undid was very recent, and has probably not been used. (I'll double-check this!)

https://wearezeta.atlassian.net/browse/WPB-27169

Checklist

Add a new entry in an appropriate subdirectory of changelog.d
Read and follow the PR guidelines

Copilot

Pull request overview

This PR (WPB-27169) changes the meaning/shape of “team size” so that API-visible teamSize represents paid seats (regular users) while apps and collaborators are reported separately, and propagates that change through Galley/Brig, the indexed user store, journaling, and tests.

Changes:

Redefines TeamSize to {teamSize, apps, collaborators} and updates JSON/OpenAPI schema plus golden fixtures.
Updates Galley/Brig logic and integration tests to treat apps/collaborators as non-members for get-team-size.
Extends indexed user store implementations and team journaling/protobuf definitions to carry separate app/collaborator counts.

Reviewed changes

Copilot reviewed 21 out of 21 changed files in this pull request and generated 5 comments.

Show a summary per file

File	Description
services/galley/src/Galley/API/Teams.hs	Adjusts team-size bookkeeping and LegalHold-related checks to the new `TeamSize` shape.
services/galley/src/Galley/API/LegalHold/Team.hs	Updates LegalHold activation size check to use the new `TeamSize` representation.
services/brig/test/integration/API/Team.hs	Updates expected `TeamSize` JSON payload in Brig integration tests.
libs/wire-subsystems/test/unit/Wire/MockInterpreters/IndexedUserStore.hs	Updates mock team-size computation and adds collaborator counting.
libs/wire-subsystems/src/Wire/UserSubsystem/Interpreter.hs	Updates max-team-size checks to use `teamSize` (paid seats) only.
libs/wire-subsystems/src/Wire/TeamJournal.hs	Updates journal event payload generation for new app/collaborator fields.
libs/wire-subsystems/src/Wire/IndexedUserStore/ElasticSearch.hs	Updates ES-based team-size result mapping to the new `TeamSize` fields.
libs/wire-api/test/golden/testObject_TeamSize_1.json	Refreshes golden JSON to new `TeamSize` schema.
libs/wire-api/test/golden/testObject_TeamSize_2.json	Refreshes golden JSON to new `TeamSize` schema.
libs/wire-api/test/golden/testObject_TeamSize_3.json	Refreshes golden JSON to new `TeamSize` schema (large-number case).
libs/wire-api/test/golden/testObject_Event_meeting_update_manual_1.json	Golden fixture update (field ordering/structure adjustment).
libs/wire-api/test/golden/testObject_Event_meeting_member_add_manual_2.json	Golden fixture update (field ordering/structure adjustment).
libs/wire-api/test/golden/testObject_Event_meeting_member_add_manual_1.json	Golden fixture update (field ordering/structure adjustment).
libs/wire-api/test/golden/testObject_Event_meeting_delete_manual_1.json	Golden fixture update (field ordering/structure adjustment).
libs/wire-api/test/golden/testObject_Event_meeting_create_manual_1.json	Golden fixture update (field ordering/structure adjustment).
libs/wire-api/test/golden/Test/Wire/API/Golden/Manual/TeamSize.hs	Updates manual golden Haskell objects for `TeamSize`.
libs/wire-api/src/Wire/API/Team/Size.hs	Redefines `TeamSize` and updates its schema/Arbitrary instance.
libs/types-common-journal/proto/TeamEvents.proto	Changes TeamEvents event-data fields for app/collaborator counts.
integration/test/Test/TeamCollaborators.hs	Adds integration assertion that collaborators are counted separately in team size.
integration/test/Test/Apps.hs	Adds integration assertion that apps are counted separately in team size.
changelog.d/1-api-changes/WPB-27169-do-not-count-apps-and-collaborators-as-members-in-get-team-size	Documents the API schema change for `get-team-size` and TeamEvents protobuf.

Suppressed comments (1)

services/galley/src/Galley/API/Teams.hs:1082

The LegalHold size check now sums teamSize + apps + collaborators here, but ensureNotTooLargeToActivateLegalHold (and the PR intent of “do not count apps/collaborators as members”) suggests only teamSize should be relevant. This inconsistency will make activation vs. later member/app changes apply different limits; please align the definition of what counts toward the LegalHold fanout size across all call sites.

    pure case uType of
      UserTypeFilterRegular -> n {teamSize = n.teamSize + 1}
      UserTypeFilterApp -> n {apps = n.apps + 1}
  ensureNotTooLargeForLegalHold tid (sizeAfterAdd.teamSize + sizeAfterAdd.apps + sizeAfterAdd.collaborators)

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

blackheaven · 2026-08-26T15:34:10Z

@@ -1,5 +1,5 @@
 {
-    "teamSize": 0,


Is it an intended breaking change?

Was previously UserSubsystem, but since it TeamCollaboratorsSubsystem is also used outside of Brig, that is not always available. Further changes: - Support BrigAPIAccess locally in Brig. - Change collaborator field type in UserDoc to collapse `Nothing` and `Just []` (remove the Maybe).

(I don't understand what it is about, and there is no author to ask.)

(This end-point only updates searchability settings, nothing else.)

This is only where we don't expect to use it. if we're wrong about this, a warning will be logged.

New schema: `{"teamSize": num, "apps": num, "collaborators": num}` (non-overlapping).

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Aug 13, 2026

fisx force-pushed the WPB-27169-do-not-count-apps-and-collaborators-as-members-in-get-team-size branch from 9bbc19f to 5f5d7d0 Compare August 14, 2026 06:52

fisx force-pushed the WPB-27169-do-not-count-apps-and-collaborators-as-members-in-get-team-size branch from 5f5d7d0 to 7c986b7 Compare August 26, 2026 14:58

fisx marked this pull request as ready for review August 26, 2026 14:58

fisx requested review from a team as code owners August 26, 2026 14:58

fisx requested a lite review from Copilot August 26, 2026 14:58

Copilot started reviewing on behalf of fisx August 26, 2026 14:59 View session

Copilot AI reviewed Aug 26, 2026

View reviewed changes

fisx commented Aug 26, 2026

View reviewed changes

Comment thread services/galley/src/Galley/API/Teams.hs Outdated

blackheaven reviewed Aug 26, 2026

View reviewed changes

fisx requested a review from blackheaven August 27, 2026 09:36

fisx force-pushed the WPB-27169-do-not-count-apps-and-collaborators-as-members-in-get-team-size branch from 6832456 to c9c245d Compare August 27, 2026 13:28

fisx and others added 16 commits August 27, 2026 15:41

Changelog.

34da921

Failing integration test.

7a7c746

Include team collaborators in contact search.

65a81ce

Remove stray TODO.

6ed241f

(I don't understand what it is about, and there is no author to ask.)

Remove bogus TODO.

65e1cdc

(This end-point only updates searchability settings, nothing else.)

Make BrigAPIAccess.Local interpreter fall back on RPC.

7076c0a

This is only where we don't expect to use it. if we're wrong about this, a warning will be logged.

Test users collaborating with more than one team.

dc66547

make sanitize-pr

b228fdb

Changelog, failing integration tests.

83768bd

[drive-by] regenerated wire-api golden tests.

cb3d0ae

Do not count apps and collaborators as members in get-team-size.

0a378bd

New schema: `{"teamSize": num, "apps": num, "collaborators": num}` (non-overlapping).

Guard legalhold team size limit when adding collaborators.

66579f8

Refactor: move legalhold helpers to TeamSubsystem.

77f3240

Fix: team size query in ES.

fd87357

Polish swagger docs.

7a33606

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

fisx force-pushed the WPB-27169-do-not-count-apps-and-collaborators-as-members-in-get-team-size branch from c9c245d to 7a33606 Compare August 27, 2026 14:07

fisx marked this pull request as draft August 27, 2026 14:31

Navigation Menu

Uh oh!

[WPB-27169] Do not count apps and collaborators as members in get-team-size. - #5445

[WPB-27169] Do not count apps and collaborators as members in get-team-size.#5445
fisx wants to merge 16 commits into
developfrom
WPB-27169-do-not-count-apps-and-collaborators-as-members-in-get-team-size

fisx commented Aug 13, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

blackheaven Aug 26, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

fisx commented Aug 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Checklist

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

blackheaven Aug 26, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

fisx commented Aug 13, 2026 •

edited

Loading